Skip to content

Tidy up plex.py#106

Open
pbrubeck wants to merge 11 commits intomainfrom
pbrubeck/fix/mg
Open

Tidy up plex.py#106
pbrubeck wants to merge 11 commits intomainfrom
pbrubeck/fix/mg

Conversation

@pbrubeck
Copy link
Collaborator

@pbrubeck pbrubeck commented Mar 12, 2026

Tidy up plex.py.

The current mechanism to convert a plex into netgen breaks for CSG geometries, (we can construct a mesh but are unable to snap the vertices to the geometry). Here we simply throw a ValueError if the provided geometry is not OCCGeometry.

This also fixes a big bubu that was in the non JITed kernel

Signed-off-by: uzerbianati <zerbinati@maths.ox.ac.uk>
@UZerbinati
Copy link
Collaborator

UZerbinati commented Mar 19, 2026

I'm really happy with this ! It was much needed, thank you @pbrubeck :)
@jorgensd I see we are breaking fenics-nightly ! Is this an issue on our side ?
@pbrubeck Can you fix the linting ?
Once we linting is fixed and we have green light from FEniCSx I'm happy to merge.

@jorgensd
Copy link
Collaborator

Its due to:
https://github.com/FEniCS/dolfinx/pull/4097/changes
I'll fix this.

@pbrubeck pbrubeck requested a review from UZerbinati March 24, 2026 17:04
@pbrubeck pbrubeck force-pushed the pbrubeck/fix/mg branch 11 times, most recently from c0fda51 to 7360aa1 Compare March 25, 2026 18:30
@pbrubeck pbrubeck force-pushed the pbrubeck/fix/mg branch 3 times, most recently from 28b953c to ec56bde Compare March 25, 2026 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants